Skip to content

DOCS-528 - Install Docusaurus Faster #4898

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 27 commits into from
Closed

DOCS-528 - Install Docusaurus Faster #4898

wants to merge 27 commits into from

Conversation

kimsauce
Copy link
Collaborator

@kimsauce kimsauce commented Dec 23, 2024

Purpose of this pull request

  • Install Docusaurus Faster to reduce the build times and memory consumption.
  • Apply edits to areas that don't adhere to new, stricter syntax that comes with this installation

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/issues/DOCS-528

@kimsauce kimsauce self-assigned this Dec 23, 2024
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Dec 23, 2024
@kimsauce kimsauce added the doc:site tech Site/tool maintenance, dependencies, updates label Dec 23, 2024
@jpipkin1
Copy link
Collaborator

@kimsauce, the "security/snyk" test failed. Do we need to have Snyk installed?

@kimsauce
Copy link
Collaborator Author

kimsauce commented Dec 23, 2024

Thanks for flagging this, @jpipkin1 . I've opened an IT ticket since that test is coming from the global Sumo Logic GitHub checks.

Update: Reached out to SOC Team for assistance.

@kimsauce kimsauce requested review from a team as code owners December 27, 2024 05:52
…h-aws-cloudformation/automatic-installation-script.md
@kimsauce kimsauce added the do-not-merge Pull requests that should not be merged label Dec 27, 2024
@kimsauce
Copy link
Collaborator Author

@jpipkin1 I was granted Snyk access. The PR check is blocked because it's throwing an error saying lightningCSS has low vulnerabilities. This is not true as it was thoroughly tested and autoinstalled by Docusaurus to support Faster. Discussing with the SOC team if it's advisable for me to override the block.

@kimsauce
Copy link
Collaborator Author

kimsauce commented Jan 5, 2025

moved to #4928

@kimsauce kimsauce closed this Jan 5, 2025
@kimsauce kimsauce deleted the faster branch February 1, 2025 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged doc:site tech Site/tool maintenance, dependencies, updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants